Skip to content

Add Chromia Vector DB#545

Merged
AruneshSingh merged 10 commits intosuperlinked:mainfrom
Prem95:main
May 20, 2025
Merged

Add Chromia Vector DB#545
AruneshSingh merged 10 commits intosuperlinked:mainfrom
Prem95:main

Conversation

@Prem95
Copy link
Copy Markdown
Contributor

@Prem95 Prem95 commented Apr 1, 2025

This pull request adds Chromia's Vector DB to the vector database comparison table

@Prem95 Prem95 requested a review from a team as a code owner April 1, 2025 15:20
Copy link
Copy Markdown
Contributor

@svonava svonava left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your submission - I left a few comments and in general, it would be good to point to source_url for all "full" and "partial" values.

Comment thread docs/tools/vdb_table/data/chromia.json Outdated
Comment thread docs/tools/vdb_table/data/chromia.json
Comment thread docs/tools/vdb_table/data/chromia.json
Comment thread docs/tools/vdb_table/data/chromia.json Outdated
Comment thread docs/tools/vdb_table/data/chromia.json Outdated
"vector_dims": {
"value": 2000,
"unlimited": false,
"source_url": "",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this limit mentioned somewhere that we can link here?

Copy link
Copy Markdown
Contributor Author

@Prem95 Prem95 Apr 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on pgvector and the max dimension is 2000 (https://github.com/pgvector/pgvector)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it - so chromia is pgvector-powered? You can mention this in the comment here / link to pgvector.

Copy link
Copy Markdown
Contributor

@svonava svonava Apr 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually - given this fact, wouldn't it make sense to reference pgvector in the other fields (filtering, hybrid search) etc? Right now, the whole chromia record is very not-filled in. Either put none to all the features that are not supported, or match pgvector where appropriate, to make this record useful for the reader please.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Prem95 can you share your opinion about the above?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure. As of now, None seems a more reasonable approach.

@Prem95
Copy link
Copy Markdown
Contributor Author

Prem95 commented Apr 7, 2025

Hello @svonava! Could I get some updates here please? Thank you!

@Prem95
Copy link
Copy Markdown
Contributor Author

Prem95 commented Apr 9, 2025

Hi @svonava I made a small change in the name. Thank you.

@svonava
Copy link
Copy Markdown
Contributor

svonava commented Apr 10, 2025

Hello @svonava! Could I get some updates here please? Thank you!

Please see my comment in one of the threads above.

@Prem95
Copy link
Copy Markdown
Contributor Author

Prem95 commented Apr 14, 2025

Updated it @svonava! Thanks :)

@Prem95
Copy link
Copy Markdown
Contributor Author

Prem95 commented May 6, 2025

Hi @svonava Please check here, thank you!

Copy link
Copy Markdown
Contributor

@AruneshSingh AruneshSingh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Prem95 Left 2 clarifying comments, let me know what you think, and then I'll merge.

Comment thread docs/tools/vdb_table/data/chromia.json
Comment thread docs/tools/vdb_table/data/chromia.json Outdated
@Prem95
Copy link
Copy Markdown
Contributor Author

Prem95 commented May 13, 2025

Hi @AruneshSingh please do review! Thank you

@AruneshSingh AruneshSingh merged commit 3380b0d into superlinked:main May 20, 2025
2 checks passed
@AruneshSingh
Copy link
Copy Markdown
Contributor

Thanks @Prem95. Merged.

@Prem95
Copy link
Copy Markdown
Contributor Author

Prem95 commented May 20, 2025

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants